Skip to content

Conversation

@vincbeck
Copy link
Contributor

Resolves #56614.

Back-port of #55506.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@vincbeck vincbeck force-pushed the vincbeck/refresh_3_1 branch from 5d2077f to 1f656d4 Compare November 24, 2025 22:23
Copy link
Member

@pierrejeambrun pierrejeambrun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be backported? Original PR is marked for 3.2

@potiuk
Copy link
Member

potiuk commented Nov 24, 2025

Should this be backported? Original PR is marked for 3.2

We discussed it in #56614 (comment) and I think the problem is that refreshing keycloak token does not work without it in 3.1.*, so @vincbeck attempted to back-port it.

@potiuk
Copy link
Member

potiuk commented Nov 24, 2025

Successfully it seems

Copy link
Contributor

@bugraoz93 bugraoz93 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Vincent!

@bugraoz93
Copy link
Contributor

It shouldn't impact as long as providers implements refresh logic which wasn't the case for 3.1

@potiuk potiuk merged commit 6b9f3cd into apache:v3-1-test Nov 25, 2025
86 checks passed
@potiuk
Copy link
Member

potiuk commented Nov 25, 2025

Changed milestone of the original PR to 3.1.4

@vincbeck vincbeck deleted the vincbeck/refresh_3_1 branch January 9, 2026 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:API Airflow's REST/HTTP API area:providers area:UI Related to UI/UX. For Frontend Developers. provider:keycloak type:bug-fix Changelog: Bug Fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants